Skip to content

Conversation

@erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented Oct 29, 2021

@erlend-aasland
Copy link
Contributor Author

It would be nice to have a native english speaker verify these changes.

Erlend Egeberg Aasland and others added 2 commits October 31, 2021 09:40
Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a solid improvement, but here are a few thoughts. (I'm not a sqlite expert by any means, but I am a native English speaker, and have a masters in journalism :)

@erlend-aasland
Copy link
Contributor Author

FYI, @AlexWaygood and @jacobtylerwalls. See also the topic Using personal or passive tone in the docs on Discourse.

@erlend-aasland erlend-aasland changed the title bpo-45677: Avoid addressing the reader as 'you' in sqlite3 docs bpo-45677: Reword first section of sqlite3 docs Nov 3, 2021
@willingc willingc merged commit 6c5a312 into python:main Nov 15, 2021
@miss-islington
Copy link
Contributor

Thanks @erlend-aasland for the PR, and @willingc for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

@bedevere-bot
Copy link

@willingc: Please replace # with GH- in the commit message next time. Thanks!

@bedevere-bot
Copy link

GH-29566 is a backport of this pull request to the 3.10 branch.

@bedevere-bot
Copy link

GH-29567 is a backport of this pull request to the 3.9 branch.

@erlend-aasland erlend-aasland deleted the sqlite-doc-you/part1 branch November 15, 2021 22:31
@erlend-aasland
Copy link
Contributor Author

Thanks, Carol!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants